Skip to content

feat(ci): semi-automate code-server-aur update #5462

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 18, 2022
Merged

feat(ci): semi-automate code-server-aur update #5462

merged 4 commits into from
Aug 18, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Aug 16, 2022

Description

This automates part of the release process for updating code-server AUR package. After the npm package is published, it will run this job which will update some files and open a PR to code-server-aur.

Changes

  • add new aur job to npm/brew workflow which runs on releases

Testing

I was able to test this by adding a temporary job to ci.yaml and it successfully opened a PR.

image

Next Steps

Fixes #5366

@jsjoeio jsjoeio requested a review from a team August 16, 2022 15:49
@jsjoeio jsjoeio changed the title first test wip: automate aur step Aug 16, 2022
@jsjoeio jsjoeio marked this pull request as draft August 16, 2022 15:49
@codecov
Copy link

codecov bot commented Aug 16, 2022

Codecov Report

Merging #5462 (7e22c1c) into main (61673b5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5462   +/-   ##
=======================================
  Coverage   72.44%   72.44%           
=======================================
  Files          30       30           
  Lines        1673     1673           
  Branches      366      366           
=======================================
  Hits         1212     1212           
  Misses        398      398           
  Partials       63       63           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61673b5...7e22c1c. Read the comment docs.

@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 15:53 Inactive
@github-actions
Copy link

github-actions bot commented Aug 16, 2022

✨ code-server dev build published to npm for PR #5462!

  • Last publish status: success
  • Commit: decd784

To install in a local project, run:

npm install @coder/code-server-pr@5462

To install globally, run:

npm install -g @coder/code-server-pr@5462

@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 16:44 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 17:00 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 17:13 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 18:20 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 18:27 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 20:49 Inactive
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 21:17 Inactive
@jsjoeio jsjoeio self-assigned this Aug 16, 2022
@jsjoeio jsjoeio added the ci Issues related to ci label Aug 16, 2022
@jsjoeio jsjoeio changed the title wip: automate aur step feat(ci): semi-automate code-server-aur update Aug 16, 2022
This adds a new job called aur which checkouts code-server-aur, updates
the files with the new version then opens a PR into the repo.
runs-on: ubuntu-latest
timeout-minutes: 10
env:
GH_TOKEN: ${{ secrets.HOMEBREW_GITHUB_API_TOKEN }}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we should rename the secret to CDRCI_GITHUB_API_TOKEN if we're going to use it in both places. I'll have to find the value or ask @deansheather again though if I do.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. For now this seems chill.

@jsjoeio jsjoeio marked this pull request as ready for review August 16, 2022 22:03
@jsjoeio jsjoeio temporarily deployed to npm August 16, 2022 22:06 Inactive
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to rename the workflow since it only mentions NPM and Brew. Maybe we can simply call it publish.yaml and even move the Docker push here.

runs-on: ubuntu-latest
timeout-minutes: 10
env:
GH_TOKEN: ${{ secrets.HOMEBREW_GITHUB_API_TOKEN }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. For now this seems chill.

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Aug 17, 2022

We might want to rename the workflow since it only mentions NPM and Brew. Maybe we can simply call it publish.yaml and even move the Docker push here.

Good idea! I'll do that in a followup PR.

@repo-ranger repo-ranger bot temporarily deployed to npm August 17, 2022 18:40 Inactive
@repo-ranger repo-ranger bot temporarily deployed to npm August 17, 2022 18:51 Inactive
@repo-ranger repo-ranger bot temporarily deployed to npm August 17, 2022 22:09 Inactive
@jsjoeio jsjoeio merged commit 9d8588b into main Aug 18, 2022
@jsjoeio jsjoeio deleted the jsjoeio/5366 branch August 18, 2022 16:22
@jsjoeio jsjoeio temporarily deployed to npm August 18, 2022 16:27 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Issues related to ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chore]: open PR into code-server-aur on publish
2 participants